Skip to content

Add URL quoting reminder to SKILL.md#13

Merged
abimaelmartell merged 1 commit intofirecrawl:mainfrom
jimmystridh:fix/url-quoting-instructions
Jan 31, 2026
Merged

Add URL quoting reminder to SKILL.md#13
abimaelmartell merged 1 commit intofirecrawl:mainfrom
jimmystridh:fix/url-quoting-instructions

Conversation

@jimmystridh
Copy link
Copy Markdown
Contributor

@jimmystridh jimmystridh commented Jan 28, 2026 •
edited
Loading

URLs with ? and & need quoting to avoid shell glob/special character interpretation.

Without the reminder, Claude Code sometimes fails on URLs with query params:

⏺ Bash(firecrawl scrape https://docs.datadoghq.com/integrations/sql-server/?tab=host -o .firecrawl/datadog-sql-server-integration.md)
  ⎿  Error: Exit code 1
     (eval):1: no matches found: https://docs.datadoghq.com/integrations/sql-server/?tab=host

After quoting it works:

⏺ Bash(firecrawl scrape "https://docs.datadoghq.com/integrations/sql_server/?tab=host" -o .firecrawl/datadog-sql-server-integration.md)
  ⎿  (No content)

@abimaelmartell
Copy link
Copy Markdown
Member

abimaelmartell commented Jan 31, 2026

good catch, thanks for contributing!

@abimaelmartell abimaelmartell merged commit 8db0ec6 into firecrawl:main Jan 31, 2026
1 check failed
@jimmystridh jimmystridh deleted the fix/url-quoting-instructions branch January 31, 2026 20:34
0x2kNJ added a commit to 0x2kNJ/cli that referenced this pull request Apr 30, 2026
@0x2kNJ
chore(prism): add integration for iNFT firecrawl#13
c479f0d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jimmystridh @abimaelmartell